Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add Sentinel to telemetry #162

Merged
merged 1 commit into from
Nov 7, 2022
Merged

Add Sentinel to telemetry #162

merged 1 commit into from
Nov 7, 2022

Conversation

glennsarti
Copy link
Contributor

It would be useful to track usage of the root sentinel config file (sentinel.hcl) to determine how many users use the extension for Sentinel.

It would be useful to track usage of the root sentinel config file (`sentinel.hcl`) to determine how many users use the extension for Sentinel.
@glennsarti glennsarti requested a review from a team as a code owner November 4, 2022 06:20
Copy link
Member

@radeksimko radeksimko left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just to help interpret the numbers, I expect that similar to Nomad, which accepts arbitrary filenames via the CLI, we can only detect the ones which follow naming conventions, correct?

i.e. Assuming the docs are true, then sentinel.hcl is just a convention, which isn't enforced, right?

Either way - great addition, thanks for the catch!

@radeksimko radeksimko added the enhancement New feature or request label Nov 4, 2022
@glennsarti
Copy link
Contributor Author

which accepts arbitrary filenames via the CLI, we can only detect the ones which follow naming conventions, correct?

That is true ... for the CLI.

One saving grace is that TFC/TFE doesn't allow you to change the root config file (sentinel.hcl or sentinel.json) so it should be more common.

@jpogran jpogran merged commit 0faec24 into hashicorp:main Nov 7, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants